Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: removed dead code and commented out unused function #144

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

saksham-gt
Copy link

- What I did

Reduced 116 warning to just 10 warning that included removal of dead code, commented out functions that are not referenced or called from anywhere in the app, made changes to variable names according to convention.
- How I did it

Initially, simple fixes were applies using dart fix --apply. This reduced the warning from 116 to 57. Then, checked out individual warnings and fixed them according to the situation.

- How to verify it

Run flutter analyze . after checking out this branch. The total number of issues should be reduced significantly.

- Description for the changelog
Significantly refactored code by removing dead code, commenting out unnecessary functions and naming variables according to convention.

@yahu1031 yahu1031 requested review from sachins-geekyants and nitesh2599 and removed request for sachins-geekyants June 28, 2022 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant